Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make ImplicitVolume updateable #228

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

claforte
Copy link
Collaborator

  • ... so density_blob_scale can be annealed over time
  • this might help in later phases

- ... so `density_blob_scale` can be annealed over time
- zero123 changes that might improve quality, and
-  ... anneal the density_blob_scale, which might help in later phases

experiment: test110_zero123_500steps_lots_of_changes
from threestudio.utils.ops import get_activation
from threestudio.utils.typing import *


@threestudio.register("implicit-volume")
class ImplicitVolume(BaseImplicitGeometry):
class ImplicitVolume(BaseImplicitGeometry, Updateable):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not needed as BaseImplicitGeometry (inherit from BaseModule) is already Updateable.

@@ -25,12 +25,12 @@ data: # threestudio/data/image.py -> SingleImageDataModuleConfig
eval_batch_size: 1
elevation_range: [-10, 80]
azimuth_range: [-180, 180]
camera_distance_range: [3.8, 3.8]
camera_distance_range: [3.8, 3.85]
Copy link
Collaborator

@voletiv voletiv Jul 12, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want these changes to be permanent? Since it's 500 iters, it will be slower than the current config. Instead, maybe add another yaml file with these changes called configs/experimental/zero123_updated.yaml?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe the extra steps and the hyperparam changes are worth it in terms of higher quality... @voletiv do you mind testing it out with a few of your images?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually... hold on, I want to run a few more tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants